Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SOK-29] Start End State #22

Merged
merged 19 commits into from
Oct 15, 2024
Merged

[SOK-29] Start End State #22

merged 19 commits into from
Oct 15, 2024

Conversation

Timur233
Copy link
Owner

### Какую задачу решаем

Скриншоты/видяшка (если есть)

image

TBD (если есть)

Экран победы отверстал, когда в игре можно будет победить, нужно будет его вывести.

- Вынес из компонента игры управление, паузой,
жизнями, началом и окончанием игры.
- Написал механику начала игры и game over
- Написал коллбэк смерти игрока
 - Модалка перехватывала нажатия клавишь
 - Передал стайте нажатия клавиш в компонент страницы
 - Обработал клики по клавишам в интерфейсе для управлением игрой
@shamemask shamemask self-requested a review October 11, 2024 15:11
Copy link
Collaborator

@VladToby VladToby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отличная работа! Выглядит круто! Замечаний нет, только предложение по улучшению.

tagName="span"
/>

<PauseHelp
className="game-controll__pause-help-buttons"
pauseIcon={
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше вынести в отдельную функцию, чтобы не усложнять логику внутри JSX
например так:

const getPauseIcon = () => {
  if (gameState.isGamePaused && gameState.isGameStarted && !gameState.isGameOver && !gameState.isGameWinning) {
    return <Icon id="arrow-right" width={12} height={16} />;
  }
  return <Icon id="pause-icon" width={16} height={17} />;
};
<PauseHelp
  className="game-controll__pause-help-buttons"
  pauseIcon={getPauseIcon()}
/>

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Готово

@shamemask shamemask marked this pull request as ready for review October 12, 2024 06:51
Copy link
Collaborator

@shamemask shamemask left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отличное решение!

Copy link

@kyzinatra kyzinatra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отличная работа! Хорошая реализация начала игры, вопросов к этой части нет

@Timur233 Timur233 merged commit 148da43 into develop Oct 15, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants